-
Notifications
You must be signed in to change notification settings - Fork 76
Remove hasLibrary guard from haddock step
#582
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Rationale: - `cabal v2-haddock` also succeeds for `executable` packages - one can always opt-out with `haddock: False` in `cabal.haskell-ci`
Atm. requires a patched version of `haskell-ci` - haskell-CI/haskell-ci#582 to include the "haddock" step. Also: - fix missing `extra-source-files:` in `.cabal` file - delete `.travis.yml`.
|
@phadej, does this change sound reasonable? |
|
Does this actually work? With: cabal-version: 2.4
name: pkkg
version: 0
executable foo
default-language: Haskell2010
main-is: exe.hs
build-depends: basemodule Main (main, foo) where
foo :: Int
foo = 42
main :: IO ()
main = print fooand % cabal --version
cabal-install version 3.6.2.0
compiled using version 3.6.2.0 of the Cabal library trying to % cabal haddock all
cabal: renderBuildTargetProblem: unexpected status (TargetDisabledByUser,Nothing)
CallStack (from HasCallStack):
error, called at src/Distribution/Client/CmdErrorMessages.hs:356:14 in main:Distribution.Client.CmdErrorMessagesI tried to find an issue in |
Seems like for your example, no haddock is actually run.
I can confirm that the error is gone on Seems like one has to pass Let me try to investigate further... (I remembered it worked fine on |
|
|
|
@phadej wrote:
Looks like it is still open: |
Yes, reading up on haskell/cabal#5890 confirms this: if you have
Since |
Also update fixtures.
bc40988 to
8db67e9
Compare
|
A few successful CI runs: |
|
Merged in #585 (please don't edit version in the future if it's release i.e. even version). |
Thanks!
Ok, understood. |
Remove
hasLibraryguard from haddock stepRationale:
cabal v2-haddockalso succeeds forexecutablepackageshaddock: Falseincabal.haskell-ciIn the wild: